3 Eerste opzet create playlist feature Patrick #11
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ik ben begonnen met het maken van de feature "create a playlist" als je klikt op de knop playlist toevoegen komt er een overlay in beeld waarbij je je nieuwe playlist een naam en afbeelding kan geven en als je dat hebt gedaan kan je klikken op create playlist. Ik heb dit gemaakt volgens PE en om het pleasurable te maken heb ik een transition toegevoegd bij het openen en sluiten van de overlay.
Wanneer er een playlist gemaakt wordt moet de playlist met de gekozen naam en afbeelding worden toegevoegd aan het lijstje met playlists maar hier ben ik nog niet aan toe gekomen.
Verder moet er gecontroleerd worden of ik me aan onze code afspraken heb gehouden en of er geen conflicten zijn.
Onze code afspraken:
Code conventies
Bij het inspringen hebben we afgesproken dat we het laten inspringen per element
Nesting css
Voor de specifiekere css maken we gebruik van nesting.
Classnames
Bij classnames maken we gebruiker van een '-' tussen worden en maken we geen gebruik van bovenkast.
https://github.com/lisavanmansom/pleasurable-ui/wiki/Week-1#code-conventies